-
Notifications
You must be signed in to change notification settings - Fork 348
feat: add scope for preheating job #3497
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3497 +/- ##
==========================================
- Coverage 51.95% 51.82% -0.13%
==========================================
Files 190 190
Lines 20302 20362 +60
==========================================
+ Hits 10547 10552 +5
- Misses 8952 9007 +55
Partials 803 803
Flags with carried forward coverage won't be shown. Click here to find out more.
|
a4e91bd
to
efebaf2
Compare
Signed-off-by: Gaius <gaius.qi@gmail.com>
524ba8f
to
3c51cb3
Compare
…e/scope Signed-off-by: Gaius <gaius.qi@gmail.com>
c354ed8
to
7ccf900
Compare
7ccf900
to
a89c7f2
Compare
a3a7997
to
50bee9b
Compare
@gaius-qi ci |
070d93e
to
4bcedd5
Compare
1351608
to
eed6609
Compare
…e/scope Signed-off-by: Gaius <gaius.qi@gmail.com>
eed6609
to
f9e524b
Compare
0e33de8
to
d818e98
Compare
…e/scope Signed-off-by: Gaius <gaius.qi@gmail.com>
d818e98
to
0f8e553
Compare
@gaius-qi could we add some e2e tests? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
Related Issue
Motivation and Context
Screenshots (if appropriate)
Types of changes
Checklist